Skip to content

Try rolldown#5482

Closed
OEvgeny wants to merge 9 commits intomainfrom
experiment/tsdown
Closed

Try rolldown#5482
OEvgeny wants to merge 9 commits intomainfrom
experiment/tsdown

Conversation

@OEvgeny
Copy link
Copy Markdown
Collaborator

@OEvgeny OEvgeny commented May 21, 2025

Fixes #

Changelog Entry

Description

Design

Specific Changes

-

  • I have added tests and executed them locally
  • I have updated CHANGELOG.md
  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

@OEvgeny OEvgeny marked this pull request as draft May 21, 2025 00:48
@OEvgeny
Copy link
Copy Markdown
Collaborator Author

OEvgeny commented May 22, 2025

Mostly works. The items left are:

  • jsx-runtime related changes
  • most of things should be marked as no external for iife

@OEvgeny OEvgeny closed this May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant